Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix copy of BankName value + BranchId/BankId #5231

Merged
merged 2 commits into from
Mar 12, 2021

Conversation

BtcContributor
Copy link
Contributor

Fixes #5200

This PR deals with cases like:

  • BankName / BranchId (as the original issue)
  • BankName / BankId

It would be trivial to add other instances if present.

Copy link
Contributor

@ripcurlx ripcurlx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NACK - Please see my comments on the naming. Also for the future please have a look at https://chris.beams.io/posts/git-commit/. I'm not a great commit message writer myself, but I think there is some potential to grow 😉

@BtcContributor
Copy link
Contributor Author

Thanks for all the feedback @ripcurlx!

@BtcContributor BtcContributor requested a review from ripcurlx March 11, 2021 16:56
Copy link
Contributor

@ripcurlx ripcurlx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

@ripcurlx ripcurlx merged commit 6148b61 into bisq-network:master Mar 12, 2021
@ripcurlx ripcurlx added this to the v1.6.0 milestone Mar 12, 2021
@BtcContributor BtcContributor deleted the bankidcopy branch March 12, 2021 09:01
Copy link

@Cha-cloud Cha-cloud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How do you connect your github and localbitcoins account together

@pazza83
Copy link

pazza83 commented May 2, 2021

Hi @Cha-cloud not sure what you are trying to do but probably best to make a post on https://bisq.community/ or ask on Keybase

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix Bank transfer 'Copy BSB number' bug
4 participants